Silence compiler warnings in non-debug builds

Declare global counter only in debug builds

ci: Add missing newline escape

Otherwise we're going to run the accessibility test suite, and it's
going to fail.

ci: Remove static build job

Now that we figured out why the build was failing on Fedora when the
profiling code was enabled, we can go back to building both shared and
static libraries by default.

ci: Use the new Fedora image

ci: Add static packages for GLib and PCRE

Otherwise Meson will warn about missing static dependencies for
sysprof-capture-3.

Fix compiler warnings in the release build

We have a bunch of debug-only variables that get set and never used
outside of debug code paths, and the compiler is not happy about it.

ci: Disable ccache

Initialise out variable

Otherwise we might return an undefined value, and the compiler will be
very cross at us.

ci: Split the static and shared builds (again)

Sadly, we can't do a static build with the profiling code enabled
because we don't have a static GLib build available.

ci: Replicate the CI image scripts from GLib

They are easier to use and document, and allow us to add new base images
in the future without making a mess.

ci: Refactor the CI pipeline

We should have a single job for building both static and shared
versions of GTK. On the other hand, having a separate job for the
release build would be a plus.

Additionally, we shouldn't use an opaque script to build GTK; the only
step in the process that requires hand-holding is running the tests
suites under Xvfb, and having the build options visible from the YAML
file gives us a better idea of what kind of build we're running.

ci: Use release builds for Flatpak bundles

tests/frame-stats.c: Include stdlib.h

It is used for exit().

Typo fix

Revert "Add the label-wrap-justify reftest to xfails"

This reverts commit 5e746d35b2c5ab8601b493d4f8ca4784328d50fa.

Turns out label-wrap-justify isn't failing in ci, and this is
now making ci fail :(

3.98.0

Add the label-wrap-justify reftest to xfails

Seems just like the other label tests, down to details
of text rendering.

Drop the toplevel-vs-popup reftest

Popups look different from toplevels. I don't think
a reftest that tries to prove otherwise has much
leg to stand on.

Disable the buttons a11y test

The test fails because radio buttons are currently broken.

Update expected test output

The a11y test for GtkAssistant produces different
output due to headerbar changes.

calendar: Fix mismatches with GDateTime

GDatetime uses 1-based month and day numbers, whereas
GktCalendars are 0-based. Correct for this.

Calendar: Make year/month/day writable again

Our a11y tests were relying on this.

testsuite: Update expected output

The output for combo boxes in a11y tests has changed
because they use popovers now.

Merge branch 'fix-gestures-test' into 'master'

Fix gestures test

See merge request GNOME/gtk!1406

Fix gestures test

This tests was testing gestures by faking an event in the
middle of a window that has a hbox with an expanding image in it.
For me (and I guess this depends on all sorts of issues like whether
CSD is enabled, font sizes, etc) the hbox ended up centered horizontally
but not vertically (probably because of csd at the top), so no events
ever hit the inner widgets.

This is fixed by emitting the events at allocation.x/y of the
hbox, which should contain both the hbox and the image (as it expands).

Merge branch 'fix-hidpi-popups' into 'master'

Fix popup placement on hidpi multi-monitor

See merge request GNOME/gtk!1405

hidpi: Rescan monitors when screen scale factor changes

All the code in e.g. init_randr15() divides the physical resolutions with
the screen scale, however if we get the screen scale from xsettings
rather than e.g. GDK_SCALE the initial setup is using the wrong value.

So, whenever the screen scale size is changed we need to trigger
a re-read of the randr data

Merge branch 'mainloop-cleanup' into 'master'

Mainloop cleanup

See merge request GNOME/gtk!1404

More updates

Merge branch 'icon-theme-api-rework' into 'master'

Icon theme api rework

See merge request GNOME/gtk!1390

Fix typo in GtkIconPaintable is-symbolic property

icon-theme: Add back and fix test for handling non-square icons

We test this by looking at the produced render nodes now that
we don't actualluy scale the icon. Also, it turns out that this
code was broken due to some typos, so we also fix those.

Expose gtk_icon_paintable_new_for_file

GtkIconPaintable: Add properties for file, icon-name and is-symbolic

Merge branch 'baselineoverlay' into 'master'

A baseline overlay

See merge request GNOME/gtk!840

icon paintable: Replace get_filename and get_resource_path with get_file()

This returns a GFile which can represent both the above.

Rename GTK_ICON_LOOKUP_LOAD_IN_THREAD to GTK_ICON_LOOKUP_PRELOAD

icontheme: Always return some icon name in get_icon_name() for lookup results

If you called gtk_icon_theme_lookup(), then always return some useful
icon name from gtk_icon_paintable_get_icon_name(), even if we picked
an unthemed icon.

Also rewrite the gtk_icon_paintable_get_icon_name() docs to make this
clearer.

Remove GTK_DEBUG=baselines

This has been reimplemented differently
in the inspector.

widget: Stop embedding baseline debugging

The inspector has an overlay that achieves the
same, in a cleaner way.

inspector: Draw baselines as an overlay

Remove unused includes

We eventually want to get rid of GtkSelectionData.
As a first step, stop including gtkselection.h unnecessarily.

docs: Mention gtk_main in the migration guide

Drop gtk_get_main_thread

This is not a very useful api, and if you need it,
you can just as easily keep track yourself which thread
called gtk_init().

Drop gtk_main and gtk_main_quit

The alternatives are to use GtkApplication,
or use GMainContext.

docs: Update an example

Don't use gtk_main() in the mainloop example.
It is going away.

Stop using gtk_main and gtk_main_quit

Replace these calls with direct use of GMainContext api.

Stop using gtk_main_quit

Stop using gtk_main and gtk_main_quit in tests and
examples. These APIs are on the way out.

Drop gtk_main_level

This api isn't useful without gtk_main, which is also
on the way out.

Drop gtk_main_iteration

This function and its cousin, gtk_main_iteration_do, are
thin wrappers around GMainContext api that should just
be used directly.

Drop gtk_events_pending

This was just a thin wrapper around gtk_main_context_pending,
which should be used directly instead.

tests: Stop using gtk_events_pending

Just use the GMainContext api directly.

win32 printing: Stop using gtk_events_pending

Just use the GMainContext API directly.

win32: One more build fix

win32: Fix the build

Merge branch 'main-cleanup' into 'master'

Cleanup in gtkmain.h

See merge request GNOME/gtk!1403

filechooser: Remove outdated hack

In GTK2, the filechooser was using a Paned, so switching between sidebar
and files list with the arrow keys didn't work (the slider would be
selected instead). So in
https://bugzilla.gnome.org/show_bug.cgi?id=161489 a crude hack was added
to make this possible.

Over the years the filechooser code has changed so that it now would do
this by default, yet the hack had been retained.

gdk: Make gdk_keyval_name() return a const char

The string is const, so return it like that.

Remove unused include

cssprovider: Remove unused include

Remove gtk_propagate_event from public API

This is not a function that can be safely called
outside of GTK.

Remove gtk_main_do_event from public API

Remove gtk_main_do_event from the docs

testsuite: Stop using gtk_main_do_event

Instead, emit ::event on the surface. Neither of
these is a proper test api, but one works as well
as the other.

docs: Remove more mentions of gtk_main_do_event

docs: Remove a reference to gtk_main_do_event

It is not great to put a function in the public api and
document it as "do not call this" only so we can refer
to its docs in other places. Therefore, fold the docs
directly into the input handling overview chapter.

docs: Remove a mention of gtk_main_do_event

css: Add a comment

All the other color values have this comment,
best to be consistent.

Merge branch 'wip/baedert/remove-outline-properties' into 'master'

css: Remove outline border radius properties

Closes #2414

See merge request GNOME/gtk!1402

gsk: Update docs sections

gsk: Small documentation additions

Fix a doc comment

Signals take two colons.

docs: Drop a few no longer existing symbols

docs: Update the switch thumbnail

Make the thumbnail for switches reflect the
current appearance of this widget.

emojichooser: Fix the docs

Make the emojichooser show up in the docs.

Fix the build with -Dgtk_doc=true

We need to export the emoji chooser symbols to
really make it public api.

Fix build with -Dgtk_doc=true

Toolbars were removed, so don't try to create
thumbnails of them.

css: Remove outline border radius properties

These are always set to the same value as the corresponding border
radius properties. They are also non-standard, so remove them and
replace them with the border radius properties everywhere.

Fixes #2414

Merge branch 'avoid-mime-sniffing' into 'master'

Avoid mime sniffing where possible

See merge request GNOME/gtk!1400

Avoid mime sniffing where possible

When we are loading symbolic pngs or svgs, we know
that we need to the png or svg loader, so there is
no need to go through (surprisingly expensive) mime
sniffing to find the right loader.

gtk-demo: Don't call gtk_main_quit

It doesn't work with GtkApplications that do not
call gtk_main().

Adwaita: Fix selection in vertical spin buttons

This was broken by more specific selectors.

Merge branch 'patch-1' into 'master'

Update migrating-3to4.xml (fixed typo, in line 377)

See merge request GNOME/gtk!1398

Update migrating-3to4.xml

label: Don't use gtk_widget_peek_style_context

We may just not have a style context. And we can just
as well go directly to the css styles.

window: Move variable initialziation just before usage

Rename gtk_widget{get,set,has}_style_class to _css_class

We want to use css instead of style everywhere now.

treepopover: Fix header guard

box: Move vfunc implementations before class_init

Just like we do everywhere else. This way we can also get rid of all the
prototypes.

calendar: Remove day-selected-double-click signal

It doesn't really make sense to treat double clicks here different than
single clicks (and is bad UX), and it also breaks switching months by
quickly trying to single-click the last/first days in the calendar.

calendar: Remove an unused enum

calendar: Remove month-changed signal

Listen to notify::month instead.

calendar: Keep a GDateTime around

Instead of a year/month/day triple.

calendar: Return a GDateTime from get_date()

calendar: Use GDateTime to select days

calendar: Use widgets for everything

calendar: Try to avoid changing size when changing the year

calendar: Remove spinning code

This is dead code

calendar: Add accessors for display properties

Instead of having a display flags enum and bitfield, just add proper
accessors for the properties, which is what we do everywhere else.